Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

use git information service on app creation and updates #75

Merged
merged 1 commit into from
Feb 5, 2024

Conversation

thirdeyenick
Copy link
Contributor

This implements the usage of the git information service into nctl. It allows to check if the specified git repository and revision can be accessed/found.

@thirdeyenick thirdeyenick force-pushed the add-initial-repo-check branch 3 times, most recently from 60c7b69 to c12ec91 Compare February 1, 2024 16:12
api/util/apps.go Outdated Show resolved Hide resolved
api/util/git_information_client.go Outdated Show resolved Hide resolved
api/util/git_information_client.go Outdated Show resolved Hide resolved
api/util/git_information_client.go Outdated Show resolved Hide resolved
api/util/git_information_client.go Outdated Show resolved Hide resolved
internal/test/apps.go Outdated Show resolved Hide resolved
create/application.go Outdated Show resolved Hide resolved
create/application.go Outdated Show resolved Hide resolved
update/application.go Outdated Show resolved Hide resolved
api/util/apps.go Outdated Show resolved Hide resolved
@thirdeyenick thirdeyenick force-pushed the add-initial-repo-check branch 4 times, most recently from e344bf9 to c4f2972 Compare February 2, 2024 15:17
This implements the usage of the git information service into `nctl`. It
allows to check if the specified git repository and revision can be
accessed/found.
@thirdeyenick thirdeyenick force-pushed the add-initial-repo-check branch from c4f2972 to 2707a7b Compare February 2, 2024 15:20
@thirdeyenick thirdeyenick merged commit 44df2a1 into main Feb 5, 2024
2 checks passed
@thirdeyenick thirdeyenick deleted the add-initial-repo-check branch February 5, 2024 09:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants